Skip to content

Adiciona testes MC/DC para o método get_address_from_cep e cobre linha não testada apontada na issue #466#532

Merged
camilamaia merged 2 commits intobrazilian-utils:mainfrom
gabriel-lima258:466
Jun 16, 2025
Merged

Adiciona testes MC/DC para o método get_address_from_cep e cobre linha não testada apontada na issue #466#532
camilamaia merged 2 commits intobrazilian-utils:mainfrom
gabriel-lima258:466

Conversation

@gabriel-lima258
Copy link
Copy Markdown
Contributor

Descrição

Este Pull Request implementa testes adicionais para o método get_address_from_cep, com foco na aplicação do critério MC/DC (Modified Condition/Decision Coverage). O objetivo é cobrir completamente todas as decisões e condições do método, incluindo a linha 172 do arquivo cep.py, anteriormente não coberta, conforme apontado na issue #466.

Mudanças Propostas

  • Adição de novos testes unitários no grupo TestGetAddressFromCEP_MC_DC, simulando:
    • CEP inválido com/sem exceção (CT1 e CT2)
    • CEP válido com resposta da API (CT3)
    • API respondendo com erro, com/sem exceção (CT4 e CT5)
    • API respondendo com {"erro": true} para cobrir a linha 172 (CT6)
  • Correção no mock de urlopen e loads para simular corretamente o contexto da API ViaCEP.
  • Cobertura aumentada de 97% para 98% no módulo cep.py.
  • Cobertura total do projeto aumentada de 98% para 99%.
  • Todos os testes passaram localmente com sucesso.

Checklist de Revisão

  • Eu li o Contributing.md
  • Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
  • Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
  • A documentação em português foi atualizada ou criada, se necessário.
  • Se feita a documentação, a atualização do arquivo em inglês.
  • Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
  • O código segue as diretrizes de estilo e padrões de codificação do projeto.
  • Todos os testes passam. Instruções
  • O Pull Request foi testado localmente. Instruções
  • Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Os testes foram organizados para facilitar a rastreabilidade e podem servir de base para testes de MC/DC em outros métodos que contenham múltiplas condições compostas.

Issue Relacionada

Closes #466

@gabriel-lima258 gabriel-lima258 requested review from a team as code owners June 15, 2025 21:56
Copy link
Copy Markdown
Member

@camilamaia camilamaia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nu, que PR mais maravilhoso! Obrigada demais @gabriel-lima258 🙇‍♀️

@camilamaia camilamaia merged commit 59d491b into brazilian-utils:main Jun 16, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adicionar cobertura de teste para linha 172 em cep.py

2 participants